Skip to content

Add compatibility for MongoConnectionDetails.getSslBundle method #2768

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

sobychacko
Copy link
Contributor

Add reflective check for the getSslBundle method in MongoConnectionDetails interface to ensure compatibility between Spring Boot 3.4.x and 3.5.x. The method exists in Spring Boot 3.5 but not in 3.4, and our code needs to work with both versions.

The implementation uses ReflectionUtils to check for the method's existence at runtime and invokes it only when available (Spring Boot 3.5+), falling back to returning null when using Spring Boot 3.4.

Thank you for taking time to contribute this pull request!
You might have already read the [contributor guide][1], but as a reminder, please make sure to:

  • Sign the contributor license agreement
  • Rebase your changes on the latest main branch and squash your commits
  • Add/Update unit tests as needed
  • Run a build and make sure all tests pass prior to submission

Add reflective check for the getSslBundle method in MongoConnectionDetails interface
to ensure compatibility between Spring Boot 3.4.x and 3.5.x. The method exists in
Spring Boot 3.5 but not in 3.4, and our code needs to work with both versions.

The implementation uses ReflectionUtils to check for the method's existence at runtime
and invokes it only when available (Spring Boot 3.5+), falling back to returning
null when using Spring Boot 3.4.

Signed-off-by: Soby Chacko <soby.chacko@broadcom.com>
@ilayaperumalg ilayaperumalg added this to the 1.0.0-RC1 milestone Apr 17, 2025
@ilayaperumalg ilayaperumalg merged commit ea3fd92 into spring-projects:main Apr 17, 2025
2 checks passed
@ilayaperumalg
Copy link
Member

Rebased and merged as ea3fd92

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants